@SpaceIm Is the patch submitted upstream wrong ? QtExcel/QXlsx#223

I like to build with C++20, and apparently there can be some issues mixing different C++ standards between libs (I forget the details). I think in the QT recipe, it uses validate() method to ensure the C++ is at least c++17, and then it is up the profile how far to push the standard.

Then cmake should use the C++xx standard requested by the profile/conan.

@jgsogo Any idea why the package is not built here ?

@MartinDelille please, read the log exposed by the CI result on the comment above.

It says:

WARN: fontconfig/2.13.93: requirement freetype/2.12.1 overridden by qt/5.15.4 to freetype/2.11.1 

So it says two of your requirements want to the same package, but different versions. Conan does not solve this kind of conflict, because it may result in runtime error, if those versions are incompatible. More information: https://docs.conan.io/en/latest/versioning/introduction.html#version-and-configuration-conflicts

There are some ways to solve it:

• Update the Qt package to use freetype/2.12.1. Need to open a new PR
• Add freetype/2.12.1 to this recipe, so Conan will solve to that version.

The second option is bad, because qxlsx does not need freetype, which would create a fake dependency.

I suggest your opening a PR for Qt package and update their dependencies first, so your error will be fixed.

@MartinDelille good news, the PR #11853 has your fix! Just need to be merged. Thanks to @ericLemanissier

@MartinDelille Sorry, that PR is only related to Qt 6. You need Qt 5. So that PR does not work for you.

@uilianries Done here: #11857

@paulharris Is my latest patch submitted upstream addressing the issue you are raising ?

Failure in build 15 (1cd379890f417a7061fe29811e64bb31ad5cbd16):

• qxlsx/1.4.3@:
  Error running command conan info qxlsx/1.4.3@#215c7ef40368a8751eaa5a749bda7a13 --json {jsonName} --dry-build -pr {profileName}:

  [settings]
  arch=x86_64
  build_type=Release
  compiler=apple-clang
  compiler.libcxx=libc++
  compiler.version=11.0
  os=Macos
  [options]
  qxlsx:shared=False
  
  ...
  WARN: libmysqlclient/8.0.25: requirement openssl/1.1.1q overridden by qt/5.15.4 to openssl/1.1.1o 
  ERROR: Conflict in cmake/3.22.5:
      'cmake/3.22.5' requires 'openssl/1.1.1q' while 'libmysqlclient/8.0.25' requires 'openssl/1.1.1o'.
      To fix this conflict you need to override the package 'openssl' in your root package.
  

^{Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.}

@paulharris Is my latest patch submitted upstream addressing the issue you are raising ?

I'll comment in there, it looks ok to me, but I wonder what the cmake experts would say about it.

but I wonder what the cmake experts would say about it.

cmake expert unfortunately I'm not <(-_-)>

I think we should wait for the next release of QXlsx because there is too much fix for Qt 6 since the last release:

[git log --oneline -- QXlsx/CMakeLists.txt]
38c1ec8 Define cpp standard only if not defined (QtExcel/QXlsx#226)
e53efc5 Use c++17 when using Qt 6
73ea291 Minor formatting fix
6d1667f Fix shared builds
1b0a279 CMake: Allow for OPTIONAL shared lib build (QtExcel/QXlsx#197)
5a086df Force strict building and fix QString usage
87afa71 set new cmake file for lib & console application

On its way: #12604